FEAT: Replace openai_objective_target initializer with explicit --target CLI parameter#1536

Merged

rlundeen2 merged 13 commits intomicrosoft:mainfrom

rlundeen2:users/rlundeen/2026_03_21_cli_target_update

Mar 24, 2026

Contributor

rlundeen2 commented Mar 24, 2026

This PR changes how objective targets are specified when running scenarios. Instead of relying on the openai_objective_target initializer to set a default target, users now explicitly select a registered target by name via --target.

Changes:

New --target flag for both pyrit_scan and pyrit_shell run commands — resolves a named target from the TargetRegistry to use as the objective target
New --list-targets command to discover available targets after initializers have run (e.g., pyrit_scan --list-targets --initializers target)
list_targets_async / print_targets_list_async added to frontend_core.py for target discovery and formatted display
Removed openai_objective_target initializer — the ScenarioObjectiveTargetInitializer and its module are deleted; references removed from init.py, .pyrit_conf_example, and all CLI help/examples
Simplified pyrit_shell startup — removed --database and --env-files args from the shell launcher (these come from the config file)
Config loading logging — added logger.info + print when loading config files for visibility
Removed unrelated stale tests in test_common_net_utility.py, test_realtime_target.py, test_supports_multi_turn.py, and test_target_capabilities.py
Added tests for list_targets_async, run_scenario_async with --target, and updated existing CLI tests to reflect the new flag

rlundeen2 added 2 commits

March 23, 2026 11:25


          adding target param to test

913a17e


          pre-commit

278899d

jsong468 reviewed

View reviewed changes

pyrit/cli/_banner.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/_cli_args.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/frontend_core.py Outdated Show resolved Hide resolved

rlundeen2 and others added 3 commits

March 24, 2026 12:11


          Update pyrit/cli/_banner.py

1c860e3

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/frontend_core.py

fcb7039

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/_cli_args.py

69782b3

Co-authored-by: jsong468 <songjustin@microsoft.com>

jsong468 reviewed

View reviewed changes

pyrit/cli/frontend_core.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/frontend_core.py Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_scan.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_scan.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_shell.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_shell.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_shell.py Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

.pyrit_conf_example Outdated Show resolved Hide resolved

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_shell.py Show resolved Hide resolved

jsong468 approved these changes

View reviewed changes

Contributor

jsong468 left a comment

lgtm! just wondering if we want get rid of some of the pyrit_scan params since they can be specified in the config file?

jsong468 reviewed

View reviewed changes

pyrit/cli/pyrit_scan.py Outdated Show resolved Hide resolved

rlundeen2 and others added 8 commits

March 24, 2026 14:36


          Update pyrit/cli/frontend_core.py

b8939ee

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/pyrit_scan.py

b3ab286

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/pyrit_scan.py

69ee03f

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/pyrit_shell.py

4fd1f94

Co-authored-by: jsong468 <songjustin@microsoft.com>


          Update pyrit/cli/pyrit_shell.py

6b8060d

Co-authored-by: jsong468 <songjustin@microsoft.com>


          pr feedback

55b0b01


          test


          Merge branch 'main' into users/rlundeen/2026_03_21_cli_target_update

57d1743

rlundeen2 merged commit ba944ee into microsoft:main

35 checks passed

rlundeen2 added a commit to rlundeen2/PyRIT that referenced this pull request


          FIX: Update E2E tests for new --target CLI and add default objective …

01c3081

…target

The E2E tests were failing with exit code 2 because:
1. --database was removed from pyrit_scan CLI (PR microsoft#1536)
2. openai_objective_target initializer was deleted (PR microsoft#1536)

Changes:
- Add DEFAULT_OBJECTIVE_TARGET tag to TargetInitializerTags
- Add openai_chat target config using generic OPENAI_CHAT_* env vars
- Tag openai_chat as the default objective target in the registry
- Update E2E test to use --target openai_chat, --initializers targets,
  and --config-file for InMemory database
- Add unit tests for default objective target tagging

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet